M3-107 만보기 인터페이스 구현 (ANDROID) #7
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
qjvk2880
approved these changes
Jun 23, 2024
lib/screens/map_screen.dart
Outdated
| return Column( | ||
| children: [ | ||
| Text('지도'), | ||
| Obx(() => Text('${AndroidStepController.steps.value}')) |
There was a problem hiding this comment.
지도 화면에 걸음수가 들어가서 conflict 날 수도 있을 것 같아요!!
koomin1227
requested changes
Jun 23, 2024
| // if(check==false){ | ||
| // _pastSteps.value = event.steps; | ||
| // print('check ok'); | ||
| // } |
lib/screens/map_screen.dart
Outdated
| Text('지도'), | ||
| ], | ||
| ); | ||
| final AndroidStepCounter AndroidStepController = |
There was a problem hiding this comment.
지도가 이미 작업되어있어서 이번 이슈에서는 하면에 표시하는 것은 안해도 될거같네요..
|
|
||
| import '../utils/walking_service.dart'; | ||
|
|
||
| class AndroidStepCounter extends GetxController implements WalkingService{ |
| import '../utils/walking_service.dart'; | ||
|
|
||
| class AndroidStepCounter extends GetxController implements WalkingService{ | ||
| late Stream<StepCount> _stepCountStream; |
There was a problem hiding this comment.
service 에서 GetX 컨트롤러를 쓰는 거 보다 controller 에서 쓰는게 좋을거 같습니다.
lib/utils/walking_service.dart
Outdated
|
|
||
| int getCurrentStep(); | ||
| void getCurrentStep(StepCount event); |
There was a problem hiding this comment.
인터페이스가 제 pr 에서도 바꿔서 충돌 날거 같네요
qjvk2880
approved these changes
Jun 23, 2024
| } | ||
|
|
||
| @override | ||
| void getCurrentStep(StepCount event){ |
There was a problem hiding this comment.
event를 listen하며 currentStep 멤버변수를 증가시키는 함수를 따로 만들고
getCurrentStep()은 해당 멤버변수를 리턴만 하게끔 하면 어떨까요?
tkdals802
force-pushed
the
feature/M3-107-android_walk_interface
branch
from
65c5938
to
be1ab12
Compare
…dler클래스를 android_notification.dart로 분리
koomin1227
force-pushed
the
feature/M3-107-android_walk_interface
branch
from
087bbc0
to
f94ef34
Compare
koomin1227
changed the title
koomin1227
approved these changes
Jul 7, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
작업 내용*
고민한 내용*
Pedometer패키지에서 사용하는 Stream에서 전체 걸음수를 받고
현재 걸음수 = 전체 걸음수 - 과거 걸음수 그리고 초기화 시킬 때마다
과거 걸음수 = 전체 걸음수로 업데이트 시켜주는 과정으로 해결하였다.
그리고 우선 map_screen의 '지도'text 아래에 걸음수가 나오도록 해놓았다.
기술적 고민
timer로 매 주기마다 계속해서 초기화 시켜 주는것
Pedometer클래스를 직접 불러와서 걸음수를 가져올 수 없었다.
구현 방안
백그라운드 실행
걸음수를 항상 측정하려면 백그라운드로 항상 동작을 해야한다. 안드로이드에서 동작 시키려면 알림센터의 위젯으로 남겨두고 작업을 실행해야한다. 이것을 구현하기 위해
flutter_foreground_task를 사용하였다.flutter_foreground_task와 메인 앱이 별도의 공간에 돌아간다. 따라서 같은 코드를 공유해도 서로 다른 메모리 상에서 구동되어 동기화가 어려웠다. 그래서 걸음수를 측정하는 로직을flutter_foreground_task에서만 돌리고 메인 앱에서 걸음수는flutter_foreground_task에서 측정되는 것을 가져오기만 하여 보여주는 역할만 하게 된다.flutter_foreground_task에서 실행되게 된다.걸음수 측정
리뷰 요구사항
스크린샷